Skip to content

Comments

Add event terms#72

Open
tomatrow wants to merge 8 commits intomainfrom
aj/add-event-terms
Open

Add event terms#72
tomatrow wants to merge 8 commits intomainfrom
aj/add-event-terms

Conversation

@tomatrow
Copy link
Contributor

No description provided.

@tomatrow tomatrow requested a review from samholmes January 30, 2026 00:43
Copy link
Contributor

@AngelCodes95 AngelCodes95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

regarding content, looks good and seems to cover a wide range. ** Not a lawyer **

</FormSection>

<CheckboxField>
<CheckboxInput
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it possible to make the required field message something other than "Please check this box if you want to proceed"? If not, then no worries.

<Paragraph>
Events may be photographed and/or recorded. These materials may capture your name,
voice, image, or likeness. By attending, you grant permission for your image to be
used in photos, videos, and other media without compensation. If you prefer not to be
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-used in photos, videos, and other media without compensation. If you prefer not to be
+used in photos, videos, and other media for marketing and promotional purposes. DEVx, reserves the right to distribute all content and materials captured or produced during the events.

Events may be photographed and/or recorded. These materials may capture your name,
voice, image, or likeness. By attending, you grant permission for your image to be
used in photos, videos, and other media without compensation. If you prefer not to be
photographed, please request an opt-out indicator at check-in.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be no opt-out. If people are disgruntled, then they can just not attend our events. This is more of a notice/disclaimer than it is asking for permission.

<SectionTitle>Data Collection</SectionTitle>
<Paragraph>
By registering, you provide certain personal information for event coordination
purposes. You may optionally consent to receive additional communications. Your data
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The data may be processed outside of the United States. We cannot necessarily guarantee that their data will be contained within the United States. Not sure if this is a worthwhile to keep. We partner with services like Luma and use AI providers that may be in the US or China, so we can't necessarily say we follow a strict data use policy if that's what's implied. We can control what we can control, but most of the data is in the public space (public as in public forum like X.com is a public forum). We don't operate in private; but when we do we may have to include a separate privacy policy.

rel="noopener noreferrer"
>
Get Your Ticket
<Button variant="primary" size="default" onClick={() => setIsModalOpen(true)}>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think instead of a modal, we should just include a sentence above saying "By checking in, you agree to the Event Terms". Make it clear it is present. Then they only need to press one button to agree.

Comment on lines 495 to 497
By submitting, I agree to these{" "}
<TermsLink href="/event-terms" target="_blank" rel="noopener noreferrer">
terms
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-By submitting, I agree to these{" "}
+By submitting, I agree to the [event terms and conditions](...)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this a mistake? @samholmes

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, the feedback is to change the wording

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should not have the modal (remove it). My main concern is about the check-in friction. I want to remove the friction and instead go with a clean disclaimer on the page.

Copy link
Contributor

@claudinethelobster claudinethelobster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work on this — clear structure and readable flow. I left a legal/content pass and recommend a few updates before merging:\n\n1) Minor policy consistency\n- Current text conflicts between "18+ or parental consent" and "under 18 must be accompanied by parent/guardian at all times".\n- Please choose one policy and make both sections match.\n\n2) Identify legal entity + contact\n- "DEVx" should be tied to a legal/organizer entity and a contact email/address for notices.\n\n3) Acceptance language\n- Add explicit enforceability text: "By checking this box and registering, you agree..." especially for waiver/release terms.\n\n4) Data Collection section needs more detail\n- Add what data is collected, retention, sharing/processors (e.g. registration/hosting platforms), privacy contact, and a privacy policy link.\n\n5) Add standard legal boilerplate\n- Effective date\n- Severability\n- Entire agreement\n- Terms updates/modification language\n\n6) Liability carve-outs\n- Current release is broad; include counsel-approved carve-outs for non-waivable rights (e.g. gross negligence/willful misconduct) where applicable.\n\n7) Photo opt-out process clarity\n- Keep opt-out language, but specify who to ask and note best-effort limitations for crowd shots.\n\n8) Cancellation/changes policy clarity\n- Add explicit policy language around event changes/cancellation and attendee-incurred costs.\n\nIf helpful, I can draft a revised v2 ToS text directly in this PR.

@claudinethelobster
Copy link
Contributor

Formatting fix (reposting cleanly):

Great work on this — clear structure and readable flow. I recommend a few updates before merging:

  1. Minor policy consistency
  • Current text conflicts between "18+ or parental consent" and "under 18 must be accompanied by parent/guardian at all times".
  • Please choose one policy and make both sections match.
  1. Identify legal entity + contact
  • "DEVx" should be tied to a legal/organizer entity and a contact email/address for notices.
  1. Acceptance language
  • Add explicit enforceability text: "By checking this box and registering, you agree..." especially for waiver/release terms.
  1. Data Collection section needs more detail
  • Add what data is collected, retention, sharing/processors (e.g. registration/hosting platforms), privacy contact, and a privacy policy link.
  1. Add standard legal boilerplate
  • Effective date
  • Severability
  • Entire agreement
  • Terms updates/modification language
  1. Liability carve-outs
  • Current release is broad; include counsel-approved carve-outs for non-waivable rights (e.g. gross negligence/willful misconduct) where applicable.
  1. Photo opt-out process clarity
  • Keep opt-out language, but specify who to ask and note best-effort limitations for crowd shots.
  1. Cancellation/changes policy clarity
  • Add explicit policy language around event changes/cancellation and attendee-incurred costs.

If helpful, I can draft a revised v2 ToS text directly in this PR.

@tomatrow
Copy link
Contributor Author

@samholmes I think I've addressed feedback

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants